Skip to content

Implementert henting av sangtekst via useQuery #43

Jump to bottom
Merged
merged 1 commit into from
Sep 10, 2026
Merged

Implementert henting av sangtekst via useQuery #43

merged 1 commit into from
Sep 10, 2026
+18 −1

Conversation

ellahw
Copy link
Member

@ellahw ellahw commented Sep 9, 2026 •
edited
Loading

Endringer

  • Lagt til custom hook useLyrics som bruker useQuery for å hente sangtekst fra lyrics.ovh
  • queryKey basert på artist + tittel gir caching og henter kun sangteksten som faktisk trengs
  • Viser sangteksten i SongCard basert på valgt sang

Test

  • npm run lint fungerer uten feil
  • npm run build fungerer uten feil
  • Åpne appen med npm run dev og:
    • Test at sangtekst vises for valgt sang
    • Test at bytte til annen sang henter ny tekst
    • Merk: noen sanger har kort/manglende tekst pga. begrensninger i
      lyrics.ovh sin database (forventet, håndteres videre i Håndter loading- og error-state #18 )

Closes #21

@ellahw ellahw requested a review from a team as a code owner September 9, 2026 14:34
@ellahw ellahw linked an issue Sep 9, 2026 that may be closed by this pull request
Hent sangtekst fra API via useQuery #21
Closed
2 tasks
mariehd
mariehd approved these changes Sep 9, 2026
View reviewed changes
Copy link
Member

@mariehd mariehd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Funker som det skal! Teksten er satt opp veldig komprimert, men det blir layout fix på slutten tenker jeg

@ellahw ellahw merged commit 793f51c into main Sep 10, 2026
@ellahw ellahw deleted the 21-hent-sangtekst-fra-api-via-usequery branch September 10, 2026 05:37
Sign in to join this conversation on GitHub.
Reviewers

@mariehd mariehd mariehd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Hent sangtekst fra API via useQuery
2 participants
@ellahw @mariehd